Add the new test cases, update the SERVLET_VERSION suffix to point to the newly published images#18410
Merged
trask merged 2 commits intoApr 30, 2026
Conversation
… the newly published images
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Liberty smoke test coverage to include the newly published OpenLiberty 26.0.0.3 servlet images, and bumps the shared servlet smoke-test image tag suffix so tests pull the latest published images from GHCR.
Changes:
- Add new
@AppServertest variants for OpenLiberty26.0.0.3across multiple JDKs (including OpenJ9 variants) inLibertySmokeTest. - Add a servlet-only Liberty smoke test variant for OpenLiberty
26.0.0.3(JDK 11). - Update
TestImageVersions.SERVLET_VERSIONto the new published servlet image tag suffix.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| smoke-tests/src/test/java/io/opentelemetry/smoketest/appserver/LibertySmokeTest.java | Adds OpenLiberty 26.0.0.3 test matrix entries (JDK 8/11/17/21/25, HotSpot + OpenJ9). |
| smoke-tests/src/test/java/io/opentelemetry/smoketest/appserver/LibertyServletOnlySmokeTest.java | Adds servlet-only coverage for OpenLiberty 26.0.0.3 on JDK 11. |
| smoke-tests/src/test/java/io/opentelemetry/smoketest/TestImageVersions.java | Bumps the shared servlet image tag suffix used by all servlet smoke-test images. |
trask
reviewed
Apr 29, 2026
Comment on lines
+46
to
+49
| /* | ||
| * OpenLiberty 21.0.0.12 | ||
| */ | ||
|
|
Member
There was a problem hiding this comment.
(and same below)
Suggested change
| /* | |
| * OpenLiberty 21.0.0.12 | |
| */ |
Member
|
hey @Channyboy, also check CI, looks like a real failure (as opposed to the sporadic variety) |
trask
reviewed
Apr 30, 2026
Channyboy
force-pushed
the
UpdateOpenLibertySmokeTests26Part2
branch
from
bb45273 to
eaca483
Compare
…they have been updated in mappings
Channyboy
force-pushed
the
UpdateOpenLibertySmokeTests26Part2
branch
from
eaca483 to
29cac57
Compare
trask
approved these changes
Apr 30, 2026
Contributor
|
Thank you for your contribution @Channyboy! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Second part of #17779 and #18344